Skip to content

gql resolver for internal freshness policy #29319

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Apr 16, 2025

Conversation

anuthebananu
Copy link
Contributor

@anuthebananu anuthebananu commented Apr 16, 2025

Summary & Motivation

Adds a resolvers on GrapheneAssetNode for internalFreshnessPolicy

How I Tested These Changes

unit tests / bk.

Changelog

Insert changelog entry or delete this section.

Copy link
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@anuthebananu anuthebananu changed the title wip - add freshness state to asset node graphql gql resolvers for asset freshness state and internal freshness policies Apr 16, 2025
@anuthebananu anuthebananu changed the title gql resolvers for asset freshness state and internal freshness policies gql resolvers for asset freshness state and internal freshness policy Apr 16, 2025
Copy link

github-actions bot commented Apr 16, 2025

Deploy preview for dagit-core-storybook ready!

✅ Preview
https://dagit-core-storybook-nqghx960p-elementl.vercel.app
https://anurag-freshness-resolver-on-asset-node.core-storybook.dagster-docs.io

Built with commit 2907d2b.
This pull request is being automatically deployed with vercel-action

@anuthebananu anuthebananu marked this pull request as ready for review April 16, 2025 04:34
@anuthebananu anuthebananu changed the title gql resolvers for asset freshness state and internal freshness policy gql resolver for internal freshness policy Apr 16, 2025
@anuthebananu anuthebananu force-pushed the anurag/freshness-resolver-on-asset-node branch from 070e196 to 2907d2b Compare April 16, 2025 14:42
@anuthebananu anuthebananu requested a review from salazarm April 16, 2025 14:46
Copy link
Contributor

@salazarm salazarm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

makes sense to me

Copy link
Contributor Author

anuthebananu commented Apr 16, 2025

Merge activity

  • Apr 16, 11:28 AM EDT: A user started a stack merge that includes this pull request via Graphite.
  • Apr 16, 11:29 AM EDT: A user merged this pull request with Graphite.

@anuthebananu anuthebananu merged commit 9409665 into master Apr 16, 2025
7 checks passed
@anuthebananu anuthebananu deleted the anurag/freshness-resolver-on-asset-node branch April 16, 2025 15:29
jamiedemaria pushed a commit that referenced this pull request Apr 18, 2025
## Summary & Motivation

Adds a resolvers on `GrapheneAssetNode` for `internalFreshnessPolicy`

## How I Tested These Changes
unit tests / bk.

## Changelog

> Insert changelog entry or delete this section.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants